-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: New Feature: outline extension BuildItems #1365
base: main
Are you sure you want to change the base?
feat: New Feature: outline extension BuildItems #1365
Conversation
Fixes redhat-developer#1094 Signed-off-by: azerr <[email protected]>
Quality Gate passedIssues Measures |
@angelozerr that looks really cool, good job! Can we somehow also show the javadoc next to the class name (that explains what the build item is for)? |
Thanks!
I fear that it is not possible by using standard |
My fear is about the need to use a specific prefix for triggering the builditem search, in this cas |
@@ -0,0 +1,53 @@ | |||
package com.redhat.devtools.intellij.quarkus.psi.internal.builditems; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
header
} | ||
|
||
/** | ||
* Returns a set of all classes in the given project that extend Renarde's |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wrong doc
The main problem is that I contribute to Symbol and it is difficult to retrieve BuildItem without special characters because Symbol return Java class. For JAX-REST we use special character |
@gastaldi do you think we can merge this feature:
|
@angelozerr I agree with @fbricon's concerns here that using |
Ok it wil require more work because it will require to develop a custom UI to show Javadoc. |
@gastaldi could you elaborate on the need for this? using class search with |
@gastaldi at some point, I was thinking about using AI to generate a better all BuildItems doc:
|
As I reported in #1094, the idea is to have a search mechanism to find the intended build item while developing an extension. Although searching for We have a list of all build items with their respective javadocs in https://quarkus.io/guides/all-builditems, that's what I use when trying to find a match for a build item when developing an extension, but having it easily findable in the IDE would be easier |
@gastaldi what about introducing another plugin for Quarkus extension developers then (and not making the one for users heavier)? |
feat: New Feature: outline extension BuildItems
Fixes #1094
Here a demo with the current PR:
Here how it works:
Project and libraries
in the scope to retrieve BuildItem from the JARs&
@gastaldi what do you think about that and the demo? Is it enough? Do you need more things?